Skip to content

Conversation

@jpipkin1
Copy link
Collaborator

@jpipkin1 jpipkin1 commented Nov 3, 2025

Purpose of this pull request

This pull request replaces old image tagging with new tagging in this section:
https://www.sumologic.com/help/docs/search/

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

DOCS-1209

@jpipkin1 jpipkin1 self-assigned this Nov 3, 2025
@jpipkin1 jpipkin1 added the doc:update Revisions and updates to content label Nov 3, 2025
@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Nov 3, 2025
@jpipkin1 jpipkin1 marked this pull request as ready for review November 5, 2025 22:10
Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for doing this, @jpipkin1 !!

Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one thing! On the legacy site, we used a setting in SnagIt that applied the gray border. Like this...
Screenshot 2025-11-05 at 3 50 23 PM

Meaning the images in this PR would have two borders because of style={{border: '1px solid gray'}}. ...
Screenshot 2025-11-05 at 3 49 46 PM

Probably ok to leave as-is though because I don't think there's a way to find-all > replace and isolate just these docs.

kimsauce

This comment was marked as resolved.

@jpipkin1
Copy link
Collaborator Author

jpipkin1 commented Nov 6, 2025

Yes, @kimsauce, I noticed that. But I don't care if there are double borders, and I'm willing to bet that customers don't care either.

@jpipkin1 jpipkin1 enabled auto-merge November 6, 2025 17:48
@jpipkin1 jpipkin1 disabled auto-merge November 6, 2025 17:48
@jpipkin1 jpipkin1 requested a review from kimsauce November 6, 2025 17:49
@jpipkin1 jpipkin1 enabled auto-merge November 6, 2025 17:51
@jpipkin1 jpipkin1 added this pull request to the merge queue Nov 6, 2025
Merged via the queue into main with commit 79d1314 Nov 6, 2025
7 checks passed
@jpipkin1 jpipkin1 deleted the docs-1206-update-images-in-search branch November 6, 2025 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Contributor approved, listed in .clabot file doc:update Revisions and updates to content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants